Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Doc values support for geo points #4207

Closed
jpountz opened this issue Nov 19, 2013 · 1 comment
Closed

Doc values support for geo points #4207

jpountz opened this issue Nov 19, 2013 · 1 comment

Comments

@jpountz
Copy link
Contributor

jpountz commented Nov 19, 2013

We should add doc values support for geo points, probably via BinaryDocValues.

Similarly to #3993, the challenge is in the mappers, since the mapper needs to know all field values in order to be able to create the field instance (because there can be a single BinaryDocValues instance per document per field).

Another open question is about the encoding: storing two doubles (16 bytes) per point is probably wasteful, should we use instead a different encoding that would be more compact while keeping precision good enough? Since the range of possible values is fixed, I'm thinking that a fixed-size encoding that would map [-180,180] into [0, 2^bits_per_value] would be rather efficient. For example, if my math is correct, a bits_per_value of 24 (62.5% reduction) could give a precision of ~5m, a bits_per_value of 32 (50% reduction) would give a precision of ~20mm, and a bits_per_value of 40 (37.5% reduction) would give a precision << 1mm.

@ghost ghost assigned jpountz Nov 19, 2013
@s1monw
Copy link
Contributor

s1monw commented Nov 19, 2013

I think this is desperately needed. The general assumption of reduced precsion could also be used in the in-memory field data version! +1 to explore the possibilities here!

jpountz added a commit to jpountz/elasticsearch that referenced this issue Dec 26, 2013
This commits add doc values support to geo point using the exact same approach
as for numeric data: geo points for a given document are stored uncompressed
and sequentially in a single binary doc values field.

Close elastic#4207
brusic pushed a commit to brusic/elasticsearch that referenced this issue Jan 19, 2014
This commits add doc values support to geo point using the exact same approach
as for numeric data: geo points for a given document are stored uncompressed
and sequentially in a single binary doc values field.

Close elastic#4207
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants